-
Notifications
You must be signed in to change notification settings - Fork 365
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[FEAT BRANCH] Add argilla-sdk
project
#4891
Conversation
for more information, see https://pre-commit.ci
…lla into feat/add-new-sdk-project
b2984fd
to
3768e3b
Compare
3768e3b
to
2e31511
Compare
argilla-sdk
project
The URL of the deployed environment for this PR is https://argilla-quickstart-pr-4891-ki24f765kq-no.a.run.app |
# Repository | ||
repo_name: argilla-io/argilla-python | ||
repo_url: https://github.com/argilla-io/argilla-python | ||
edit_uri: edit/main/docs/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I saw this line in the latest sync. Should this affect the current project setup? In that case, should we comment until we have merged the project into the default/develop/main one? cc @davidberenstein1957 @burtenshaw
I think we should review also some changes in docs: https://github.com/argilla-io/argilla/actions/runs/9352932120/job/25742223903?pr=4891#step:10:39 cc @davidberenstein1957 @sdiazlor |
This PR just applies some minimal changes for the argilla SDK docs generation: - Freeze the default branch to `feat/v2.0.0` - Launch a docs generation for docs branches starting with `docs/**` - Remove PR-related events from triggers.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Regarding the docs, looks like they are all.
Description
This PR setup the argilla-sdk project. This PR only copy the code base from the argilla-python repository.
No changes in package definition or docs have been applied yet.UPDATED: The only change is this line which raise an ruff error when applying commit hooks.Update
The code on this PR won't work with the server version on
feat/v2.0.0
until we resolve SDK tasks for the issue #4773. For the sake of the review complexity, I would prefer to tackle this migration as a separate PR.After those changes, the new SDK will still be fully compatible with argilla server 1.29.0.
Closes #4888
Closes #4895
Type of change
(Please delete options that are not relevant. Remember to title the PR according to the type of change)
How Has This Been Tested
(Please describe the tests that you ran to verify your changes. And ideally, reference
tests
)Checklist
CHANGELOG.md
file (See https://keepachangelog.com/)